Skip to content

Vtab strict auth mode and passwords with externalauth#5928

Open
mohitkhullar wants to merge 1 commit into
bloomberg:mainfrom
mohitkhullar:vtab-auth
Open

Vtab strict auth mode and passwords with externalauth#5928
mohitkhullar wants to merge 1 commit into
bloomberg:mainfrom
mohitkhullar:vtab-auth

Conversation

@mohitkhullar
Copy link
Copy Markdown
Contributor

No description provided.

Copy link
Copy Markdown

@roborivers roborivers left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cbuild submission: Error ⚠.
Regression testing: Success ✓.

The first 10 failing tests are:
truncatesc_offline_generated [failed with core dumped] **quarantined**
consumer_non_atomic_default_consumer_generated **quarantined**
simpleauth_password

Copy link
Copy Markdown

@roborivers roborivers left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cbuild submission: Success ✓.
Regression testing: Success ✓.

The first 10 failing tests are:
logfill [db unavailable at finish] **quarantined**
consumer_non_atomic_default_consumer_generated **quarantined**
reco-ddlk-sql [timeout] **quarantined**

@mohitkhullar mohitkhullar marked this pull request as ready for review May 8, 2026 20:35
Comment thread tests/simpleauth_password.test/comdb2_simple_auth_verify.out Outdated
Copy link
Copy Markdown

@roborivers roborivers left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cbuild submission: Error ⚠.
Regression testing: 0/0 tests failed ⚠.

@mohitkhullar
Copy link
Copy Markdown
Contributor Author

sqlpool_queue, logical_operations, active_osqls (from cdb2op1) will now require authorization

Copy link
Copy Markdown

@roborivers roborivers left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cbuild submission: Error ⚠.
Regression testing: Success ✓.

The first 10 failing tests are:
queuedb_rollover **quarantined**
consumer_non_atomic_default_consumer_generated **quarantined**
reco-ddlk-sql [timeout] **quarantined**

Copy link
Copy Markdown

@roborivers roborivers left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cbuild submission: Error ⚠.
Regression testing: Success ✓.

The first 10 failing tests are:
logfill [db unavailable at finish] **quarantined**
queuedb_rollover_noroll1_generated **quarantined**
consumer_non_atomic_default_consumer_generated **quarantined**
reco-ddlk-sql [timeout] **quarantined**

@mohitkhullar mohitkhullar requested a review from mponomar May 20, 2026 14:45
mponomar
mponomar previously approved these changes May 20, 2026
Copy link
Copy Markdown

@roborivers roborivers left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cbuild submission: Success ✓.
Regression testing: Success ✓.

The first 10 failing tests are:
sc_resume_logicalsc_generated **quarantined**
sc_resume
consumer_non_atomic_default_consumer_generated **quarantined**
simpleauth_password
reco-ddlk-sql [timeout] **quarantined**

Signed-off-by: mkhullar <mohit.khullar@gmail.com>
Copy link
Copy Markdown

@roborivers roborivers left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cbuild submission: Success ✓.
Regression testing: Success ✓.

The first 10 failing tests are:
logfill [db unavailable at finish] **quarantined**
sc_truncate_multiddl_generated [db unavailable at finish] **quarantined**
consumer_non_atomic_default_consumer_generated **quarantined**

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants